feat(cli): add interactive setup wizard and modernize init command by cameroncooke · Pull Request #247 · getsentry/XcodeBuildMCP

cameroncooke · 2026-02-27T22:14:00Z

Add xcodebuildmcp setup — an interactive terminal wizard that configures project defaults (project/workspace, scheme, simulator, workflows, debug mode, Sentry opt-out) and persists results to .xcodebuildmcp/config.yaml.

Previously users had to hand-edit YAML to configure their project. The setup wizard discovers available projects, schemes, and simulators, presents them in a polished terminal UI via @clack/prompts, and writes a complete config file with a summary of what changed.

Supporting changes:

Shared Prompter abstraction (src/cli/interactive/prompts.ts) — wraps @clack/prompts with a testable interface that auto-selects defaults in non-interactive mode
sentryDisabled promoted to first-class config key — previously env-var-only (XCODEBUILDMCP_SENTRY_DISABLED), now settable in config.yaml and resolved through the standard config layering. A new hydrateSentryDisabledEnvFromProjectConfig() bridges the config file to the env var before Sentry init in all three entrypoints (cli, daemon, mcp server)
MCP tool refactors — extracted reusable discoverProjects(), listSchemes(), and listSimulators() functions from the MCP *Logic wrappers so both the MCP layer and CLI can call them directly. No behavior change to MCP tool responses
init command modernized — switched from raw readline to @clack/prompts, added interactive skill type and client selection when no flags are provided, replaced Cursor/Codex targets with generic "Agents Skills" target, added ~ expansion for --dest
persistProjectConfigPatch() — new utility for atomic config file updates with support for mutually exclusive key cleanup (projectPath vs workspacePath)

pkg-pr-new · 2026-02-27T22:15:29Z

Open in StackBlitz

npm i https://pkg.pr.new/xcodebuildmcp@247

commit: ceed2e1

src/cli/commands/setup.ts

src/mcp/tools/project-discovery/list_schemes.ts

src/cli/commands/setup.ts

src/cli/commands/init.ts

cursor

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Autofix Details

Bugbot Autofix prepared fixes for all 3 issues found in the latest run.

✅ Fixed: Interactive Claude selection silently skipped for MCP skill
- Added check for Claude in user-selected targets array to treat interactive selection as explicit intent, same as --client flag.
✅ Fixed: Internal key filter misses camelCase logLevel variant
- Added 'logLevel' to the internalKeys set to filter out the camelCase variant created by yargs camel-case-expansion.
✅ Fixed: Inconsistent TTY check in ensureAgentsGuidance after refactor
- Changed guard from process.stdin.isTTY to isInteractiveTTY() to match the check used by promptConfirm function.

Or push these changes by commenting:

@cursor push 3034395277

Preview (3034395277)

diff --git a/src/cli/commands/init.ts b/src/cli/commands/init.ts
--- a/src/cli/commands/init.ts
+++ b/src/cli/commands/init.ts
@@ -286,7 +286,7 @@
   }
 
   if (!force) {
-    if (!process.stdin.isTTY) {
+    if (!isInteractiveTTY()) {
       throw new Error(
         `${AGENTS_FILE_NAME} exists and requires confirmation to update. Re-run with --force to apply the change in non-interactive mode.`,
       );
@@ -665,6 +665,11 @@
     return { allowedTargets: targets, skippedClients: [] };
   }
 
+  const hasClaudeTarget = targets.some((t) => t.id === 'claude');
+  if (clientFlag === 'claude' || hasClaudeTarget) {
+    return { allowedTargets: targets, skippedClients: [] };
+  }
+
   const allowedTargets: ClientInfo[] = [];
   const skippedClients: Array<{ client: string; reason: string }> = [];
 
@@ -679,9 +684,5 @@
     allowedTargets.push(target);
   }
 
-  if (clientFlag === 'claude') {
-    return { allowedTargets: targets, skippedClients: [] };
-  }
-
   return { allowedTargets, skippedClients };
 }

diff --git a/src/cli/register-tool-commands.ts b/src/cli/register-tool-commands.ts
--- a/src/cli/register-tool-commands.ts
+++ b/src/cli/register-tool-commands.ts
@@ -162,7 +162,16 @@
 
       // Convert CLI argv to tool params (kebab-case -> camelCase)
       // Filter out internal CLI options before converting
-      const internalKeys = new Set(['json', 'output', 'style', 'socket', 'log-level', '_', '$0']);
+      const internalKeys = new Set([
+        'json',
+        'output',
+        'style',
+        'socket',
+        'log-level',
+        'logLevel',
+        '_',
+        '$0',
+      ]);
       const flagArgs: Record<string, unknown> = {};
       for (const [key, value] of Object.entries(argv as Record<string, unknown>)) {
         if (!internalKeys.has(key)) {

_{This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.}

src/cli/commands/init.ts

src/cli/register-tool-commands.ts

src/cli/commands/init.ts

src/utils/sentry-config.ts

src/cli/commands/init.ts

src/utils/logger.ts

docs/dev/debug-flag-investigation-2026-02-28.md

cameroncooke · 2026-03-02T21:17:01Z

Re: investigation notes committed as project documentation — agreed, removed in 693a6f9. These were ephemeral investigation notes, not permanent documentation.

src/cli/interactive/prompts.ts

src/cli/commands/daemon.ts

Add `xcodebuildmcp setup` — an interactive terminal wizard that walks users through configuring project defaults (project/workspace, scheme, simulator, workflows, debug mode, Sentry opt-out) and persists the result to .xcodebuildmcp/config.yaml. Key changes: - New setup command with clack-based interactive prompts - Shared Prompter abstraction for testable TTY/non-interactive prompts - Promote sentryDisabled from env-var-only to first-class config key - Extract reusable functions from discover_projs, list_schemes, list_sims so both MCP tools and CLI can call them directly - Modernize init command to use clack prompts and interactive selection - Replace Cursor/Codex client targets with generic Agents Skills target - Add persistProjectConfigPatch for atomic config file updates

Extract shared isInteractiveTTY() to prompts.ts and add try/catch to withSpinner so the spinner is stopped if the task throws, preventing garbled terminal output on error.

…malizeLogLevel Align internal log level naming with Sentry SDK conventions ('warn' instead of 'warning'). Add normalizeLogLevel() to safely map external level strings (including MCP protocol's 'warning') to internal LogLevel values. Also removes the CLI daemon logLevel passthrough (debug flag no longer overrides daemon log level) and filters 'log-level' from tool command args.

… debug Pass existing project config into workflow option evaluation so that debug-gated workflows (e.g. doctor) appear in the setup wizard when the user's config already has debug: true.

Return structured JSON from init install/uninstall flows when no TTY is available. Keep clack-based output for interactive sessions. This gives agent and CI callers a stable machine-readable contract without introducing additional flags or output modes.

Capture AGENTS guidance outcomes during non-interactive init and embed the status in the emitted JSON report. Include path and error details when AGENTS updates fail so automation can reason about partial success without parsing stderr. Preserve existing failure behavior by throwing after JSON emission on AGENTS errors, and extend init command tests to cover created, updated, and error statuses in JSON output.

Preserve backwards compatibility for existing scripts that pass --log-level warning while keeping help output focused on the internal warn level token. Map warning to warn during argument coercion in both the main and lightweight yargs app builders so init/setup and normal CLI execution behave consistently.

Delete AI-generated investigation log that was accidentally committed as project documentation. The file contained stale line-number references and phase-by-phase working notes, not permanent docs.

Add the same empty-options check to createNonInteractivePrompter's selectOne that the TTY prompter already has, preventing an out-of-bounds access if called with an empty array.

The --log-level option already coerces 'warning' to 'warn' for backward compatibility, but --daemon-log-level was missing the same coercer, causing a yargs validation error for users passing the old value.

The setup wizard requires explicit --client or --dest in non-interactive mode. Add --client auto to the test so it exercises the auto-detect policy path, and emit skip reasons before the zero-targets error.

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

src/utils/sentry-config.ts

cursor bot reviewed Feb 27, 2026

View reviewed changes

src/cli/commands/setup.ts Outdated Show resolved Hide resolved

src/mcp/tools/project-discovery/list_schemes.ts Show resolved Hide resolved

src/cli/commands/setup.ts Show resolved Hide resolved

cursor bot reviewed Feb 27, 2026

View reviewed changes

src/cli/commands/setup.ts Show resolved Hide resolved

cursor bot reviewed Feb 28, 2026

View reviewed changes

src/cli/commands/setup.ts Show resolved Hide resolved

src/cli/commands/init.ts Outdated Show resolved Hide resolved

cameroncooke force-pushed the feat/setup-wizard branch from e00170d to 25e5a8b Compare March 2, 2026 12:32

cursor bot reviewed Mar 2, 2026

View reviewed changes

src/cli/commands/init.ts Show resolved Hide resolved

src/cli/commands/init.ts Show resolved Hide resolved

src/cli/register-tool-commands.ts Outdated Show resolved Hide resolved

sentry bot reviewed Mar 2, 2026

View reviewed changes

src/cli/commands/init.ts Outdated Show resolved Hide resolved

cursor bot reviewed Mar 2, 2026

View reviewed changes

src/utils/sentry-config.ts Show resolved Hide resolved

src/cli/commands/init.ts Outdated Show resolved Hide resolved

cursor bot reviewed Mar 2, 2026

View reviewed changes

src/utils/logger.ts Show resolved Hide resolved

docs/dev/debug-flag-investigation-2026-02-28.md Outdated Show resolved Hide resolved

cursor bot reviewed Mar 2, 2026

View reviewed changes

src/cli/interactive/prompts.ts Show resolved Hide resolved

cursor bot reviewed Mar 2, 2026

View reviewed changes

src/cli/commands/daemon.ts Show resolved Hide resolved

cameroncooke added 14 commits March 2, 2026 22:17

fix(cli): deduplicate isInteractiveTTY and stop spinner on error

36c89fd

Extract shared isInteractiveTTY() to prompts.ts and add try/catch to withSpinner so the spinner is stopped if the task throws, preventing garbled terminal output on error.

🐛 fix(setup): show debug-gated workflows when existing config enables…

84eba49

… debug Pass existing project config into workflow option evaluation so that debug-gated workflows (e.g. doctor) appear in the setup wizard when the user's config already has debug: true.

📝 docs: add debug flag investigation and reorder example config fields

18b09a4

fix: address init policy and argv filtering review feedback

31c5572

fix: respect runtime sentry disable and simplify init policy flow

0222b9d

remove ephemeral debug-flag investigation notes

d240011

Delete AI-generated investigation log that was accidentally committed as project documentation. The file contained stale line-number references and phase-by-phase working notes, not permanent docs.

fix(cli): guard non-interactive prompter against empty options

a73d3e2

Add the same empty-options check to createNonInteractivePrompter's selectOne that the TTY prompter already has, preventing an out-of-bounds access if called with an empty array.

fix(cli): add warning-to-warn coercer for --daemon-log-level

af96d9d

The --log-level option already coerces 'warning' to 'warn' for backward compatibility, but --daemon-log-level was missing the same coercer, causing a yargs validation error for users passing the old value.

fix(init): adapt MCP auto-install test for interactive wizard flow

7c97e9d

The setup wizard requires explicit --client or --dest in non-interactive mode. Add --client auto to the test so it exercises the auto-detect policy path, and emit skip reasons before the zero-targets error.

cameroncooke force-pushed the feat/setup-wizard branch from 8dd1462 to 7c97e9d Compare March 2, 2026 22:27

cursor bot reviewed Mar 2, 2026

View reviewed changes

src/utils/sentry-config.ts Show resolved Hide resolved

cameroncooke merged commit d94b96a into main Mar 2, 2026
9 checks passed

cameroncooke deleted the feat/setup-wizard branch March 2, 2026 22:46

Uh oh!

Conversation

cameroncooke commented Feb 27, 2026

Uh oh!

pkg-pr-new bot commented Feb 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cameroncooke commented Mar 2, 2026

Uh oh!

Uh oh!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

pkg-pr-new bot commented Feb 27, 2026 •

edited

Loading

cursor bot left a comment •

edited

Loading